Skip to content

Add save matches to separate files, start/end line numbers#33

Merged
vearutop merged 2 commits intomasterfrom
sp2
Dec 17, 2025
Merged

Add save matches to separate files, start/end line numbers#33
vearutop merged 2 commits intomasterfrom
sp2

Conversation

@vearutop
Copy link
Member

No description provided.

@github-actions
Copy link

Lines Of Code

Language Files Lines Code Comments Blanks Complexity Bytes
Go 8 (+1) 1503 (+62) 1149 (+45) 41 (+1) 313 (+16) 308 31.5K (+1.2K)
Go (test) 3 74 62 0 12 10 1.7K (+13B)
Markdown 5 232 (+27) 186 (+21) 0 46 (+6) 0 8.7K (+869B)
YAML 7 543 (+4) 474 (+4) 34 35 0 18K (+55B)

@github-actions
Copy link

Go API Changes

# github.com/bool64/progress/cmd/catp/catp
## compatible changes
Options.VersionLabel: added

# summary
Inferred base version: v0.3.22
Suggested version: v0.4.0

@github-actions
Copy link

Unit Test Coverage

total: (statements) 37.8%
changed lines: (statements) 58.5%, coverage is less than 90.0%, consider testing the changes more thoroughly

Coverage of changed lines
File Function Coverage
Total 58.5%
cmd/catp/catp/app.go 55.3%
cmd/catp/catp/app.go:26 Main 69.1%
cmd/catp/catp/catp.go 67.4%
cmd/catp/catp/catp.go:202 scanFile 90.8%
cmd/catp/catp/catp.go:312 cat 33.3%
cmd/catp/catp/filter.go 53.3%
cmd/catp/catp/filter.go:67 addFilter 100.0%
cmd/catp/catp/filter.go:91 shouldWrite 100.0%
cmd/catp/catp/filter.go:52 saveTo 0.0%
cmd/catp/catp/writer.go 52.0%
cmd/catp/catp/writer.go:13 makeWriter 46.6%
Coverage diff with base branch
File Function Base Coverage Current Coverage
Total 35.7% 37.8% (+2.1%)
cmd/catp/catp/app.go Main 48.1% 50.0% (+1.9%)
cmd/catp/catp/catp.go cat 59.4% 73.7% (+14.3%)
cmd/catp/catp/catp.go scanFile 57.1% 59.3% (+2.2%)
cmd/catp/catp/filter.go saveTo no function 0.0%
cmd/catp/catp/writer.go makeWriter no function 52.0%

@vearutop vearutop merged commit 670b2ac into master Dec 17, 2025
5 checks passed
@vearutop vearutop deleted the sp2 branch December 17, 2025 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant